-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Created a testcase for auto primary content #460
base: dev
Are you sure you want to change the base?
Conversation
Deploying with Cloudflare Pages
|
Codecov Report
@@ Coverage Diff @@
## dev #460 +/- ##
=======================================
Coverage 91.76% 91.76%
=======================================
Files 33 33
Lines 4834 4834
Branches 32 32
=======================================
Hits 4436 4436
Misses 384 384
Partials 14 14 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall one test case is not enough to test the whole feature. Please rethink you approach and submit your changes.
coll.map((c) => { | ||
c.lessons.map((lesson) => { | ||
if (lesson.content.length == 1) | ||
expect(lesson.content[0].primary).toBe(true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are expecting the lesson's first content to be primary if the array length is 1.
- What is this test cases checking if the content length is more then 1?
- Is this test case covering creating content with a primary conflict?
- Is this test case covering updating content to be primary where there already is a primary in the Array?
No description provided.